Skip to content

Fix atomic error on loongarch64 #2536

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix atomic error on loongarch64 #2536

wants to merge 1 commit into from

Conversation

zhaixiaojuan
Copy link

When I build the project on a Debian loongarch64 platform, I get an error:

Finished <<< controller_manager_msgs [1.27s]
--- stderr: hardware_interface
/usr/bin/ld: /opt/ros/jazzy/lib/libpal_statistics.so: undefined reference to `__atomic_store_16'
/usr/bin/ld: /opt/ros/jazzy/lib/libpal_statistics.so: undefined reference to `__atomic_load_16'
/usr/bin/ld: /opt/ros/jazzy/lib/libpal_statistics.so: undefined reference to `__atomic_compare_exchange_16'
collect2: error: ld returned 1 exit status

So submit this PR to fix this problem.

Related issue：#2518

@zhaixiaojuan zhaixiaojuan mentioned this pull request Sep 11, 2025
Open
saikishor
saikishor requested changes Sep 11, 2025
View reviewed changes
Copy link
Member

@saikishor saikishor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue seems to be in pal_statistics right?
At least from the log I feel so. Maybe the fix should go there instead

@zhaixiaojuan
Copy link
Author

The issue seems to be in pal_statistics right? At least from the log I feel so. Maybe the fix should go there instead

Hi, I have submitted the relevant PR to pal_statistics( pal-robotics/pal_statistics#20 ). I tested this locally, and after fixing only this issue in pal_statistics, I still got the error : "/opt/ros/jazzy/lib/libpal_statistics.so: undefined refer to `__atomic_load_16'" when building the project.

@christophfroehlich
Copy link
Contributor

The issue seems to be in pal_statistics right? At least from the log I feel so. Maybe the fix should go there instead

Hi, I have submitted the relevant PR to pal_statistics( pal-robotics/pal_statistics#20 ). I tested this locally, and after fixing only this issue in pal_statistics, I still got the error : "/opt/ros/jazzy/lib/libpal_statistics.so: undefined refer to `__atomic_load_16'" when building the project.

it still uses the lib from /opt/ros/jazzy. maybe does the overlay from your workspace not work, can you uninstall it and rebuild it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saikishor saikishor saikishor requested changes

@anfemosa anfemosa Awaiting requested review from anfemosa

@ARK3r ARK3r Awaiting requested review from ARK3r

@bmagyar bmagyar Awaiting requested review from bmagyar

@fmauch fmauch Awaiting requested review from fmauch

@LukasMacha97 LukasMacha97 Awaiting requested review from LukasMacha97

@mamueluth mamueluth Awaiting requested review from mamueluth

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zhaixiaojuan @christophfroehlich @saikishor